[Refactor] 새로운 Intent 구조 적용 #400

PeraSite · 2025-10-17T07:51:32Z

Summary

하드코딩된 문자열을 Key로 사용하고, 값이 잘 들어왔기를 바래야하는 Intent 구조를 개선합니다.

getParcelableExtra와 putExtra(String, Parcelable)를 사용해 직렬화에서 오는 오버헤드는 있겠습니다만 코드 퀄리티가 크게 개선됩니다.

Describe your changes

Intent Extra가 필요한 Activity 내부에 Extra 값을 표현하는 Parcelable data class와 ActivityCompanion를 상속받는 companion object를 선언합니다.

@AndroidEntryPoint
class IntroActivity : AppCompatActivity() {
    @Parcelize
    data class IntentOptions(val launchPath: LaunchPath) : Parcelable
    companion object : ActivityCompanion<IntentOptions>(IntroActivity::class, IntentOptions::class)
}

이후 Activity 내부에서 Intent 값을 참조할 때는 다음과 같은 방식을 사용합니다.

val launchPath = intentOptions?.launchPath ?: LaunchPath.ICON
//                     ^ IntroActivity.IntentOptions?

새 Intent를 만들어서 startActivity할 때는 다음과 같은 방식을 사용합니다.

IntroActivity.start(context, IntroActivity.IntentOptions(LaunchPath.WIDGET)) {
    addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
}

기존 Intent.apply {}로 적용하던 flag 처리도 한번에 할 수 있습니다.

Activity 내부에서는 여전히 Intent 값이 없을 경우에 대응해야 하지만, 외부에서는 무조건 IntentOptions을 만들어야하기 떄문에 훨씬 깔끔하다고 생각합니다!

Issue

Resolves 하드 코딩된 Intent Extra 개선 #374

To reviewers

현재 구조를 모든 Activity에 적용하면 좋을까요?
Fragment를 만들 때 사용하는 arguments, Bundle.put() 도 개선하겠습니다!

gemini-code-assist · 2025-10-17T07:51:50Z

Summary of Changes

Hello @PeraSite, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 Intent에 하드코딩된 문자열 키를 사용하여 데이터를 전달하는 기존 방식을 개선하기 위해 새로운 Intent 구조를 도입합니다. ActivityCompanion이라는 추상 클래스를 통해 Parcelable 데이터 클래스를 활용하여 Intent 데이터를 타입 안전하게 전달하고 수신할 수 있도록 합니다. 이 변경으로 인해 런타임 오류 가능성이 줄어들고 코드 가독성이 향상됩니다. 현재는 IntroActivity에만 이 새로운 구조를 시범적으로 적용했습니다.

Highlights

새로운 Intent 구조 도입: 하드코딩된 문자열 키 대신 타입 안전한 ActivityCompanion 클래스를 도입하여 Intent 데이터를 전달하고 수신하는 방식을 개선했습니다.
IntroActivity에 새로운 구조 적용: IntroActivity에 새로운 Intent 구조를 시범적으로 적용하여 Parcelable 데이터 클래스를 통해 launchPath를 전달하도록 변경했습니다.
코드 간결성 및 안정성 향상: Intent 생성 및 플래그 처리 방식이 간결해졌으며, 런타임 오류 가능성을 줄여 코드의 안정성을 높였습니다.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

Copilot

Pull Request Overview

This PR refactors the Intent handling mechanism from hardcoded string keys to a type-safe structure using Parcelable data classes and a generic ActivityCompanion base class. The refactoring is demonstrated with IntroActivity, replacing the string-based "launch_path" extra with a structured IntentOptions approach.

Key Changes:

Introduced ActivityCompanion base class for type-safe Intent creation and parameter extraction
Applied new Intent structure to IntroActivity with IntentOptions data class
Updated all IntroActivity launch points (widget, local notifications, remote notifications) to use the new API

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
`ActivityCompanion.kt`	New base class providing type-safe Intent creation and options extraction
`IntroActivity.kt`	Adopted new Intent structure with `IntentOptions` data class and simplified launch path handling
`MealWidget.kt`	Updated to use `IntroActivity.start()` with type-safe `IntentOptions`
`NotificationReceiver.kt`	Migrated local notification Intent creation to new structure
`EatSsuFirebaseMessagingService.kt`	Migrated remote notification Intent creation to new structure
`launchApp.kt`	Removed deprecated widget launch utility function

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

app/src/main/java/com/eatssu/android/presentation/base/ActivityCompanion.kt

gemini-code-assist

Code Review

안녕하세요! Intent 구조를 개선하기 위한 리팩토링 PR 잘 보았습니다. 하드코딩된 문자열 키를 사용하는 대신 Parcelable 데이터 클래스와 ActivityCompanion 패턴을 도입하여 타입 안정성을 크게 향상시킨 점이 매우 인상적입니다. 이 새로운 구조는 코드의 가독성을 높이고 잠재적인 런타임 오류를 줄이는 데 큰 도움이 될 것입니다.

PR 설명에서 질문주신 것처럼, 이 구조를 다른 모든 Activity에도 점진적으로 적용하는 것을 적극 추천합니다. 일관성 있는 구조는 프로젝트 전체의 유지보수성을 향상시킬 것입니다.

한 가지 개선 제안을 ActivityCompanion.kt 파일에 남겼습니다. 확인 부탁드립니다.

app/src/main/java/com/eatssu/android/presentation/base/ActivityCompanion.kt

…t 적용

Copilot

Pull Request Overview

Copilot reviewed 28 out of 28 changed files in this pull request and generated 2 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

app/src/main/java/com/eatssu/android/presentation/mypage/userinfo/UserInfoActivity.kt

...rc/main/java/com/eatssu/android/presentation/cafeteria/review/modify/ModifyReviewActivity.kt

HI-JIN2

한줄 한줄 읽어보았습니다!

먼저 제훈님의 고민과 노고에 대해서 감사드립니다

제 감상평은 일단 아름답습니다!!

하지만 한가지 걱정이 되는건, 사람이 계속 교체되고, 제훈님도 이 레포에 기여하는 것을 멈췄을때에 유지보수가 될 수 있을까? 라는 생각이 조금 들어요. 그래서 이러한 intent 확장함수를 만들어 사용하는 것이 보편적인 안드로이드 엔지니어가 행하는 방법인가를 조금 고민해보아야할 것 같아유...

게다가 컴포즈 네비게이션으로 점차 넘어가게 되면 intent의 사용비율이 되게 많이 줄어들거라서 한 프로젝트 안에서 엄청 반복되는 코드가 아닌데 이걸 확장함수로 만들어야하나 라는 생각도 조금은 들구요

위의 코멘트와 별개로, 제가 보기에도 이 코드는 마음에 들지 않아서 상수화는 적극 도입하면 좋을 것 같습니다!

 val intent = Intent(this, IntroActivity::class.java).apply {
            putExtra("launch_path", "remote_notification")

PeraSite · 2025-10-20T08:25:32Z

한줄 한줄 읽어보았습니다!

먼저 제훈님의 고민과 노고에 대해서 감사드립니다

제 감상평은 일단 아름답습니다!!

하지만 한가지 걱정이 되는건, 사람이 계속 교체되고, 제훈님도 이 레포에 기여하는 것을 멈췄을때에 유지보수가 될 수 있을까? 라는 생각이 조금 들어요. 그래서 이러한 intent 확장함수를 만들어 사용하는 것이 보편적인 안드로이드 엔지니어가 행하는 방법인가를 조금 고민해보아야할 것 같아유...

게다가 컴포즈 네비게이션으로 점차 넘어가게 되면 intent의 사용비율이 되게 많이 줄어들거라서 한 프로젝트 안에서 엄청 반복되는 코드가 아닌데 이걸 확장함수로 만들어야하나 라는 생각도 조금은 들구요

위의 코멘트와 별개로, 제가 보기에도 이 코드는 마음에 들지 않아서 상수화는 적극 도입하면 좋을 것 같습니다!
 val intent = Intent(this, IntroActivity::class.java).apply {
            putExtra("launch_path", "remote_notification")

진님께서 우려스럽다고 말씀해주신 부분에 대해서 저도 충분히 고민해보았는데요,
코틀린 언어만의 companion object 상속이나 Receiver 확장 함수 등 기술성 복잡도와 난이도가 필요한 수정이라고 생각해요.
추후에 수정하는 사람이 이 부분에 대해서 잘 알고 있지 못하다면 각 Activity마다 선언되어있는 이상한 코드를 보면서 이게 무슨 의미지? 할 것 같아요.
그럼에도 안전하게 Intent를 만들어낼 수 있는 방향성 자체는 저도 굉장히 필요하다고 생각해서 ActivityCompanion에 최대한 주석을 잘 달아보겠습니다.
별도 문서를 만드는 것보단 코드 내에서 바로 왜 필요했는지, 어떻게 쓰면 되는지를 알면 좋으니까요!

캄포즈로 모든 액티비티를 다 마이그레이션하는 것이 말씀해주신 것처럼 최고의 방법임은 틀림없으나 상당히 오랜 시간이 걸려야 마이그레이션이 진행 될 것 같아요. 그 전까지 새로운 뷰를 만들면서 Intent extra를 새롭게 설계해야할 수도 있으니 미리미리 진행해두면 좋을 것 같습니다!

refactor: IntroActivity 새로운 Intent 구조 적용

95e1da5

PeraSite self-assigned this Oct 17, 2025

PeraSite marked this pull request as draft October 17, 2025 07:52

PeraSite requested review from HI-JIN2, Copilot and kangyuri1114 and removed request for HI-JIN2, Copilot and kangyuri1114 October 17, 2025 07:52

Copilot AI reviewed Oct 17, 2025

View reviewed changes

app/src/main/java/com/eatssu/android/presentation/base/ActivityCompanion.kt Outdated Show resolved Hide resolved

gemini-code-assist bot reviewed Oct 17, 2025

View reviewed changes

app/src/main/java/com/eatssu/android/presentation/base/ActivityCompanion.kt Outdated Show resolved Hide resolved

PeraSite added 15 commits October 17, 2025 16:56

chore: companion object private 변경, androidx.core.content.IntentCompa…

ccea94a

…t 적용

feat: ActivityCompanion, ActivityCompanionWithArgs 분리

76a1f20

refactor: WebViewActivity 새 Intent 구조 적용

7786e8e

refactor: ForceUpdateDialogActivity 새 Intent 구조 적용

e54505f

refactor: LoginActivity 새 Intent 구조 적용

27029a2

refactor: MainActivity 새 Intent 구조 적용

c79d3d1

chore: ActivityUtil.kt startActivity deprecate

f9373c6

feat: 기본값으로 만들 수 있는 ActivityCompanionWithArgsDefault 추가

f3cbe9a

refactor: SignOutActivity 새 Intent 구조 적용

2b05847

refactor: WebViewActivity 호출 구조 변경

9e2860d

refactor: DeveloperActivity 새 Intent 구조 적용

07a00fa

refactor: MyReviewListActivity 새 Intent 구조 적용

6e96014

refactor: AndroidMessageDialogActivity 새 Intent 구조 적용

a03b597

chore: Default start 추가

a9a8bc9

refactor: UserInfoActivity 새 Intent 구조 적용

a096d62

PeraSite added 5 commits October 17, 2025 17:43

refactor: ReportActivity 새 Intent 구조 적용

18e60af

refactor: ModifyReviewActivity 새 Intent 구조 적용

3371d2e

refactor: ReviewActivity 새 Intent 구조 적용

b6f4318

refactor: ReviewWriteRateActivity 새 Intent 구조 적용

20db1c5

refactor: type-safe한 Fragment arguments

dec1a51

PeraSite marked this pull request as ready for review October 17, 2025 08:58

PeraSite requested a review from Copilot October 17, 2025 08:59

Copilot AI reviewed Oct 17, 2025

View reviewed changes

app/src/main/java/com/eatssu/android/presentation/mypage/userinfo/UserInfoActivity.kt Show resolved Hide resolved

...rc/main/java/com/eatssu/android/presentation/cafeteria/review/modify/ModifyReviewActivity.kt Show resolved Hide resolved

HI-JIN2 reviewed Oct 20, 2025

View reviewed changes

PeraSite added 3 commits October 20, 2025 17:33

Merge branch 'develop' into refactor/typesafe-intent

37ea8ea

docs: ActivityCompanion, FragmentCompanion 주석 추가

5a12705

feat: FragmentCompanionWithArgsDefault 추가

49eabf7

PeraSite requested a review from HI-JIN2 October 20, 2025 09:03

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Refactor] 새로운 Intent 구조 적용 #400

[Refactor] 새로운 Intent 구조 적용 #400

PeraSite commented Oct 17, 2025 •

edited

Loading

Uh oh!

gemini-code-assist bot commented Oct 17, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

HI-JIN2 left a comment

Uh oh!

PeraSite commented Oct 20, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

[Refactor] 새로운 Intent 구조 적용 #400

Are you sure you want to change the base?

[Refactor] 새로운 Intent 구조 적용 #400

Conversation

PeraSite commented Oct 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Describe your changes

Issue

To reviewers

Uh oh!

gemini-code-assist bot commented Oct 17, 2025

Summary of Changes

Highlights

Footnotes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

HI-JIN2 left a comment

Choose a reason for hiding this comment

Uh oh!

PeraSite commented Oct 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

PeraSite commented Oct 17, 2025 •

edited

Loading

PeraSite commented Oct 20, 2025 •

edited

Loading